New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Added missing endpoints and fixed delete endpoints /w auth #4

Merged

guacamole merged 1 commit into master from missing-endpoints

Jul 10, 2021

Member

jay-dee7 commented Jul 10, 2021

Signed-off-by: jay-dee7 jasdeepsingh.uppal@gmail.com

Get Manifest endpoint was deleted accidentally, so this PR adds it back
We now close req bodies immediately after reading them
Added Delete (Image, Layer) endpoints, but they don't delete anything yet (since deleting is kind of tricky on SkyNet)


          Added missing endpoints and fixed delete endpoints /w auth

695046d

Signed-off-by: jay-dee7 <jasdeepsingh.uppal@gmail.com>

jay-dee7 requested a review from guacamole

July 10, 2021 12:52

jay-dee7 self-assigned this

guacamole approved these changes

View reviewed changes

Member

guacamole left a comment

LGTM 💯

guacamole merged commit dc6477d into master

guacamole deleted the missing-endpoints branch

July 10, 2021 12:57

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet